Skip to content

Fix tag dropdown not finalizing selection - #13490

Open
abdosaeedelhassan wants to merge 1 commit into
keepassxreboot:developfrom
abdosaeedelhassan:fix/gui-entry-fixes
Open

Fix tag dropdown not finalizing selection#13490
abdosaeedelhassan wants to merge 1 commit into
keepassxreboot:developfrom
abdosaeedelhassan:fix/gui-entry-fixes

Conversation

@abdosaeedelhassan

Copy link
Copy Markdown

When selecting a tag from the completer dropdown, the selection now immediately finalizes into a tag pill instead of requiring an extra action.

Fixes #13444

[skip ci] Fix 2 of 10 — GUI entry fixes

Screenshots

Testing strategy

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)
  • ✅ New feature (change that adds functionality)
  • ✅ Breaking change (causes existing functionality to change)
  • ✅ Refactor (significant modification to existing code)
  • ✅ Documentation (non-code change)

@varjolintu varjolintu added the pr: ai-assisted Pull request contains significant contributions by generative AI label Jul 3, 2026
When selecting a tag from the completer dropdown, the selection
now immediately finalizes into a tag pill instead of requiring
an extra action.

Fixes keepassxreboot#13444

[skip ci] Fix 2 of 10 — GUI entry fixes
Comment on lines +265 to +269
void PasswordWidget::setPasswordStrength(const QString& password)
{
updatePasswordStrength(password);
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not part of this change

Comment thread src/gui/PasswordWidget.h
void selectAll();
void setReadOnly(bool state);
void setEchoMode(QLineEdit::EchoMode mode);
void setPasswordStrength(const QString& password);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Comment on lines +1027 to +1029
if (entry->isAttributeReference("Password")) {
m_mainUi->passwordEdit->setPasswordStrength(entry->resolveMultiplePlaceholders(entry->password()));
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: Tags pr: ai-assisted Pull request contains significant contributions by generative AI ux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Selecting Tag in dropdown does not "add" it completely

3 participants